Skip to content

Validate final fork lifecycles and record review gates - #30

Draft
mtwilliams wants to merge 1 commit into
codex/ortex-scheduler-evidencefrom
codex/ortex-final-validation
Draft

mtwilliams wants to merge 1 commit into
codex/ortex-scheduler-evidencefrom
codex/ortex-final-validation

Conversation

@mtwilliams

@mtwilliams mtwilliams commented Oct 29, 2026 •

Copy link
Copy Markdown
Collaborator

Completes the integration ledger and lifecycle diagnostic for the fork-only stack. The final soak checks golden scores through 100,000 ordinary calls, 1,000 alternating file/memory load-disposal cycles, and 100,000 managed bounded calls per runtime. The harness now discards results with Enum.each: an earlier collecting comprehension retained an accumulator and distorted memory observations. Temporary native lifetime counters confirmed reclamation and were removed; no production native implementation changed in this PR.

Corrected local results: bounded RSS ranged 102.8–104.2 MB on runtime 1.17.3 and 105.3–106.5 MB on 1.24.4; process counts stayed at 124 at every sample. Every call matched golden scores. These are synthetic CPU diagnostics, not leak guarantees for other models/providers. Exact byte counts and reproduction are in docs/integration.md.

The ledger records review order #2–#30, 216 suite checks (two optional ResNet cases excluded), constructor/environment/logging/bootstrap gates, scheduler measurements, fresh-VM upgrades, unsigned task commits, and ancestry-preserving merge guidance. API-25 per-run profiling and measurement-dependent GPU/NN primitives remain explicitly deferred; upstream/shared-package work stays outside scope.

Stacked on #29. Integration branch codex/ortex-integration combines the stack for source, all seven precompiled consumers, and complete package validation. The current combined stack additionally includes ownership fix #31; exact validation is recorded below. No release tag, upstream write, or Hex publication is part of this change.

Final combined validation is green at a4cef8490858d89b7827ef6e62ba652107434373 on codex/ortex-integration:

  • Source matrix: API 17/runtime 1.17.3, API 17/runtime 1.24.4, API 24/runtime 1.24.4, and both Alpine architectures.
  • Precompiled consumers and package: all seven CPU targets and complete checksum/package assembly passed with compiler execution forbidden during consumer testing. Release/tag-only jobs were intentionally skipped.
  • Local suite: 218 checks passed on both runtimes; two optional external ResNet cases excluded. Long ordinary/load-disposal/bounded diagnostics checked every golden score, and post-reservation bounded soaks retained flat process counts and bounded sampled memory.

Review in stack order #2–#31. Preserve ancestry with merge commits, or rebase/retarget the remaining stack after squash/rebase merges. The worktree is clean. Fresh VM required for native resource upgrades. No upstream writes, release tags, or Hex publication were performed.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant